Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(workflow): MWPW-161458 - Add Core Web Vitals monitoring to PR pipeline #198

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat(workflow): MWPW-161458 - Add Core Web Vitals monitoring to PR pipeline #198

wants to merge 4 commits into from

Conversation

sanrai
Copy link
Collaborator

@sanrai sanrai commented Oct 29, 2024 •
edited
Loading

Changes

  • Added new GitHub workflow job run-web-vitals-checks to monitor Core Web Vitals metrics during PR deployments
  • Implemented comprehensive user interaction simulation for accurate performance measurement
  • Added automatic testing of LCP, FID, CLS, FCP, and TTFB metrics against industry standard thresholds
  • Updated endpoint URL from chimera runtime to www.adobe.com/chimera-api
  • Added puppeteer dependency for browser automation

Testing

  • The workflow runs automatically after deployment
  • Metrics are measured against the following thresholds:
    • LCP (Largest Contentful Paint): < 2.5s
    • FID (First Input Delay): < 100ms
    • CLS (Cumulative Layout Shift): < 0.1
    • FCP (First Contentful Paint): < 1.8s
    • TTFB (Time to First Byte): < 800ms

Related Issue

[MWPW-161458]

@sanrai sanrai changed the title testt Core Web Vitals CI/CD check Oct 29, 2024
@sanrai sanrai changed the title Core Web Vitals CI/CD check feat(workflow): add Core Web Vitals monitoring to PR pipeline Nov 4, 2024
@sanrai sanrai changed the title feat(workflow): add Core Web Vitals monitoring to PR pipeline feat(workflow): MWPW-161458 - Add Core Web Vitals monitoring to PR pipeline Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sheridansunier sheridansunier sheridansunier approved these changes

@cmiqueo cmiqueo cmiqueo approved these changes

@jedjedjedM jedjedjedM Awaiting requested review from jedjedjedM

@gmirijan gmirijan Awaiting requested review from gmirijan

@shkhan91 shkhan91 Awaiting requested review from shkhan91

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sanrai @cmiqueo @sheridansunier